Repository navigation
feat: Add local flag overrides using dev server - #837
Merged
Merged
Conversation
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using default effort and found 2 potential issues.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit f1ff359. Configure here.
Overriding a flag key that hasn't been synced now serves it as a local flag instead of returning "flag not found", so code can be run against a flag before it exists upstream. Local flags get shown with a local chip and a checkbox appears for filtering them only when local flags exist. Most consumers will not see any UI differences. Also greys out the overrides filter when no overrides exist. Claude <claude@anthropic.com>
davidbrackbill
force-pushed
the
db/dev-server-override-file
branch
from
October 2, 2026 00:26
f1ff359 to
94c0d77
Compare
davidbrackbill
changed the title
davidbrackbill
marked this pull request as ready for review
October 2, 2026 00:30
zakk-verrilli-ld
approved these changes
Oct 2, 2026
zakk-verrilli-ld
left a comment
zakk-verrilli-ld
left a comment
There was a problem hiding this comment.
This is cool but why not just create the flag in Catamorphic upfront?
Contributor
Author
If your agent doesn't have that access, it can't test the code paths. If you work by making a bunch of draft PRs or iterations and then taking the best one, you'll have a lot of orphaned flags. |
davidbrackbill
added a commit
that referenced
this pull request
Oct 5, 2026
2 of 3 tasks
nieblara
added a commit
that referenced
this pull request
Oct 28, 2026
…t fix (#851) * Revert "fix: Add local flag overrides using dev server (#837) (#850)" This reverts commit 9197f92. * fix: reset the override and local flag filters when they no longer apply The filter state stayed set after the last override went away. A new override then turned the filter on again and hid other flags. Two effects now reset each filter when its condition is false. Add tests for both filters. Co-authored-by: Ramon Niebla <nieblara@users.noreply.github.com> --------- Co-authored-by: Cursor Agent <cursoragent@cursor.com> Co-authored-by: Ramon Niebla <nieblara@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Changes
Overriding a flag key before the flag exists in the upstream environment is now allowed. This is useful for agents to play around with flag values before actually creating a flag in Catamorphic.
Local flags get shown with a local chip and a checkbox appears for filtering them only when local flags exist. Most consumers will not see any UI differences.
Also greys out the overrides filter when no overrides exist.
Demo
flags.local.mov